Skip to content

fix: update environment variable name from OOSE_HOME to GOOSE_HOME in docker-compose.yml#7340

Closed
TheCrAge100 wants to merge 1 commit into
aaif-goose:mainfrom
TheCrAge100:goose-typo
Closed

fix: update environment variable name from OOSE_HOME to GOOSE_HOME in docker-compose.yml#7340
TheCrAge100 wants to merge 1 commit into
aaif-goose:mainfrom
TheCrAge100:goose-typo

Conversation

@TheCrAge100
Copy link
Copy Markdown

Summary

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

Related Issues

Relates to #ISSUE_ID
Discussion: LINK (if any)

Screenshots/Demos (for UX changes)

Before:

After:

@TheCrAge100 TheCrAge100 requested a review from a team as a code owner February 19, 2026 02:49
Copy link
Copy Markdown
Collaborator

@jh-block jh-block left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@jh-block
Copy link
Copy Markdown
Collaborator

@TheCrAge100 can you add the Signed-off-by to the commit to satisfy the DCO check? https://github.com/block/goose/blob/main/CONTRIBUTING.md#developer-certificate-of-origin

Comment thread documentation/docs/docker/docker-compose.yml
Copy link
Copy Markdown
Collaborator

@blackgirlbytes blackgirlbytes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing!

I approve but @TheCrAge100 it looks like your DCO check is failing. here's how to fix it!

After that, we will be able to merge. https://github.com/block/goose/blob/main/CONTRIBUTING.md#developer-certificate-of-origin

@DOsinga DOsinga self-assigned this Mar 9, 2026
@DOsinga
Copy link
Copy Markdown
Collaborator

DOsinga commented Mar 10, 2026

thanks @TheCrAge100 -- with the DCO missing and this being a very small change, I went ahead and merged:

#7786

instead which is the same, but we do appreciate the contribution!

@DOsinga DOsinga closed this Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants